Skip to content

fix: preserve defaultModelTier when resolving experts from API#686

Merged
FL4TLiN3 merged 3 commits intomainfrom
fix/default-model-tier-api-resolution
Mar 4, 2026
Merged

fix: preserve defaultModelTier when resolving experts from API#686
FL4TLiN3 merged 3 commits intomainfrom
fix/default-model-tier-api-resolution

Conversation

@FL4TLiN3
Copy link
Contributor

@FL4TLiN3 FL4TLiN3 commented Mar 4, 2026

Summary

  • toRuntimeExpert in both @perstack/runtime and @perstack/installer was dropping defaultModelTier when converting API response experts to runtime Expert objects
  • This caused all API-fetched experts to ignore their configured model tier and fall back to the provider's highest-tier model (e.g., claude-opus-4-6 instead of claude-haiku-4-5 for defaultModelTier: "low")
  • Also fixed configExpertToExpert in installer which had the same omission
  • Bumped @perstack/api-client to ^0.0.58 across installer, runtime, studio

Test plan

  • Existing delegation-executor tests pass (model tier resolution)
  • Added test: "preserves defaultModelTier from API response" in resolve-expert.test.ts
  • typecheck passes
  • changeset validated

🤖 Generated with Claude Code

FL4TLiN3 and others added 2 commits March 4, 2026 14:14
toRuntimeExpert in both runtime and installer packages was dropping
defaultModelTier from API-fetched experts, causing all delegates to
fall back to the provider's highest-tier model instead of respecting
their configured tier.

Also bump @perstack/api-client to ^0.0.58.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 enabled auto-merge (squash) March 4, 2026 14:17
@FL4TLiN3 FL4TLiN3 merged commit 9ebcef3 into main Mar 4, 2026
11 checks passed
@FL4TLiN3 FL4TLiN3 deleted the fix/default-model-tier-api-resolution branch March 4, 2026 14:20
@FL4TLiN3 FL4TLiN3 mentioned this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant